-
-
Notifications
You must be signed in to change notification settings - Fork 306
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: add build workflow for debug nodejs #6103
Conversation
working-directory: 'nodejs' | ||
|
||
- name: Compile the nodejs | ||
run: make -j4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to use make -j$(nproc --all)
here to default to the number of available cores?
- name: Copy nodejs build | ||
run: make install | ||
working-directory: 'nodejs' | ||
env: | ||
DESTDIR: ${{ github.workspace }}/nodejs-debug-build-${{ github.event.inputs.version }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Running make install
only moves the Release
version to the DESTDIR
.
You need to also copy nodejs/out/Debug/node
binary over the Release
binary in DESTDIR/lib/node
(pretty sure about that dest path but not 1000%)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!! 🚀
Performance Report✔️ no performance regression detected Full benchmark results
|
🎉 This PR is included in v1.13.0 🎉 |
Motivation
More flexible debugging for native nodejs modules.
Description
setup-node
also don't provide this support.Steps to test or reproduce
-Run all tests.